OsPath conversion of Annex.YouTubeDl

The change of R.doesPathExist to doesFileExist I think fixes a reversion
introduced in commit 1ceece3108f03badcca0d9c64cd287f9352656b3. Before
that commit, it was doesFileExist, and I assume to point is that this is
only supposed to return files, not any subdirectories that yt-dlp might
create while running.

OsPath conversion of Remote.Adb

Note that the additional use of System.FilePath.Posix likely fixes a
problem if this were used on windows. The AndroidPath uses / directory
separators. Before this, on windows, \ would have been used.

The change to newtype AndroidPath is only documentation.

more OsPath conversion

Sponsored-by: Leon Schuermann

more OsPath conversion (464/749)

Sponsored-by: unqueued

more OsPath conversion

Sponsored-by: Graham Spencer

replace removeLink with removeFile

removeFile calls unlink so removes anything not a directory. So these
are replaceable in order to convert to OsPath.

more OsPath conversion

Sponsored-by: Joshua Antonishen

more OsPath conversion

Sponsored-by: k0ld

more OsPath conversion

Sponsored-by: Brock Spratlen

more OsPath conversion

Sponsored-by: Jack Hill

Merge branch 'master' into ospath

update

Merge branch 'master' into ospath

Merge branch 'ospath-mk1'

Merge branch 'master' of ssh://git-annex.branchable.com

update appveyor cache directory

With old:

Cache entry not found: C:\projects\git-annex\Users\appveyor\.stack

I think fixing this will result in caching preventing a full rebuild of
git-annex each time.

response

avoid build warning

An odd one, it complained about security_attributes being defined but
not used, but it was used..

Added a comment

avoid build warnings on windows

Merge branch 'master' of ssh://git-annex.branchable.com

bring back OsPath changes

I hope that the windows test suite failure on appveyor was fixed by
updating to a newer windows there. I have not been able to reproduce
that failure in a windows 11 VM run locally.

Merge branch 'ospath-mk1' into ospath

Revert "disable OsPath build flag on windows for now"

This reverts commit 55cf9ce28fe1c8a4ee64f1843cd3a88e85bcbf35.

Problem was fixed by commit c1e90767daeff7557e047ea4229d83c4e78f9f12

fix FileIO openTempFile on Windows

When an UNC-style path is passed into openTempFile, the returned file
starts with that same style of path. Which can cause problems, eg piping
that filename to git failed. So, convert the output filename to be
relative to the input temp directory.

fix FileIO openTempFile on Windows

When an UNC-style path is passed into openTempFile, the returned file
starts with that same style of path. Which can cause problems, eg piping
that filename to git failed. So, convert the output filename to be
relative to the input temp directory.

Merge branch 'master' of ssh://git-annex.branchable.com

switch appveyor windows build to newer version

Added a comment

close

document a SNAFU

throwing away a week's work

This was an attempt at continuing the RawFilePath transition into an
OsPath transition. Unforunately, it had multiple test suite failures on
windows. Debugging those seems that it would be more work than the
minor optimisation value of this.

merge in doc changes from master

Merge branch 'master' into ospath

fix liuxstandalone build

disable OsPath build flag on windows for now

Test suite failure looks like this:

        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        fatal: could not open '\\?\C:\Users\appveyor\AppData\Local\Temp\1\hash-cc81b41d-dfda-4ae8-904b-b531742443cc' for reading: No such file or directory
        git-annex: fd:4: Data.ByteString.hGetLine: end of file
        git-annex: user error (git ["--git-dir=.git","--work-tree=.","--literal-pathspecs","hash-object","-w","--no-filters","--stdin-paths"] exited 128)

This is apparently in Git.HashObject, and probably in hashBlob, which uses a
temp file with a name starting with "hash", but I have not been able to tell
what's wrong.

I don't understand where the "\\?" path prefix (windows UNC-style path)
is coming from in the path that gets fed into git hash-file, or why git
hash-file can't open the file.

remove temp debugging code

debug test suite crash on windows

more OsPath conversion

Sponsored-by: Kevin Mueller

Merge branch 'master' into ospath

fix windows build

windows build fix

fix windows build and clarify comment

Merge branch 'master' into ospath

fix fromOsPath on windows

Same reasons as 9c9baf7c6581235643661b94f70f76f8a10ae60e

fix toOsPath on windows

After recent changes to use OsPath, test suite fails on windows with:

Exception: Uneven number of bytes: 5. This is not a Word16 bytestream.

Hopefully this fixes it. The innefficiency of the conversion is
unfortunate.

response

more OsPath conversion

Sponsored-by: Nicholas Golder-Manning

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

more OsPath conversion

Sponsored-by: Luke T. Shumaker

Merge branch 'master' of ssh://git-annex.branchable.com

Merge branch 'master' into ospath

windows build fix

and a little more bonus RawFilePath conversion

more OsPath conversion

Sponsored-by: mycroft

Merge /home/joey/tmp/git-annex into ospath

remove Read instance for AssociatedFile

This instance is not used.

correct typo

poll vote (My phone (or MP3 player))

poll vote (My phone (or MP3 player))

comment

update

update

comment

comment

Added a comment: Simple config amendment for Apache served repositories

more OsPath conversion

keyFile has a nice improvement; since a Key is a ShortByteString, it can
be converted to an OsPath without needing the copy that was done before.

Unfortunately, fileKey has to convert from a ShortByteString to a
ByteString in order to use attoparsec, and then the results get
converted back to an OsPath, so there are now 2 copies.
Maybe attoparsec will eventually get a ShortByteString API,
see https://github.com/haskell/attoparsec/issues/225

Sponsored-by: Joshua Antonishen

optimise literalOsPath

Taking a ShortByteString and using OverloadedStrings should avoid it
being converted from a String.

The reason there is no IsString instance for OsPath is presumably the
bad behavior of IsString for ByteString on unicode btw. But
literalOsPath won't be used with unicode in git-annex.

Sponsored-by: unqueued

Merge branch 'master' of ssh://git-annex.branchable.com

comment

Added a comment

Added a comment

Added a comment

thoughts

update comment

more OsPath conversion

Merge branch 'master' of ssh://git-annex.branchable.com

fix link

reporting on FTBFS

Added a comment

Added a comment

Added a comment

Some clarifications in my reproduce steps about the state of the rsync remote

rename bugs/rsyncurl_without___34____58____34___creates_local_folder_as_repo.mdwn to bugs/rsyncurl_without___34____58____34___creates_local_folder_as_remote.mdwn

Initial post

Added a comment

Added a comment

more OsPath conversion

OsPath conversion

Decent win in exportDirectories, since it operates on ShortByteString
end to end now without needing conversion. That made it worth
implementing an OsPath specific code path there.

And ExportLocation already being a ShortByteString is an good example of why
it's a good thing that OsPath uses that!

Sponsored-by: k0ld on Patreon